Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rollup of 9 pull requests #78386

Closed
wants to merge 32 commits into from

Conversation

Dylan-DPC-zz
Copy link

Successful merges:

Failed merges:

r? @ghost

RalfJung and others added 30 commits October 11, 2020 23:59
The logging in compiletest was migrated from log crate to a tracing, but
the initialization code was never changed, so logging is non-functional.

Initialize tracing subscriber using default settings.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
…fe-fn, r=Mark-Simulacrum

`#[deny(unsafe_op_in_unsafe_fn)]` in sys/wasm

This is part of rust-lang#73904.

This encloses unsafe operations in unsafe fn in `libstd/sys/wasm`.

@rustbot modify labels: F-unsafe-block-in-unsafe-fn

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
…crum

Tweak `if let` suggestion to be more liberal with suggestion and to not ICE

Fix rust-lang#77218. Fix rust-lang#77238.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
…ulacrum

transmute_copy: explain that alignment is handled correctly

The doc comment currently is somewhat misleading because if it actually transmuted `&T` to `&U`, a higher-aligned `U` would be problematic.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
…-Simulacrum

Initialize tracing subscriber in compiletest tool

The logging in compiletest was migrated from log crate to a tracing, but
the initialization code was never changed, so logging is non-functional.

Initialize tracing subscriber using default settings.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
…Mark-Simulacrum

Add issue template link to IRLO

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Tweak match arm semicolon removal suggestion to account for futures

* Tweak and extend "use `.await`" suggestions
* Suggest removal of semicolon on prior match arm
* Account for `impl Future` when suggesting semicolon removal
* Silence some errors when encountering `await foo()?` as can't be certain what the intent was

*Thanks to https://twitter.com/a_hoverbear/status/1318960787105353728 for pointing this out!*

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
…, r=m-ou-se

Capture output from threads spawned in tests

This is revival of rust-lang#75172.

Original text:
> Fixes rust-lang#42474.
>
> r? `@dtolnay` since you expressed interest in this, but feel free to redirect if you aren't the right person anymore.

---

Closes rust-lang#75172.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Fix rust-lang#78192

Check which places are marked dead.

Fixes rust-lang#78192

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Do not try to report on closures to avoid ICE

Fixes rust-lang#78262
@Dylan-DPC-zz
Copy link
Author

@bors r+ rollup=never p=5

@bors
Copy link
Collaborator

bors commented Oct 26, 2020

📌 Commit e86664e has been approved by Dylan-DPC

@bors bors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Oct 26, 2020
@bors
Copy link
Collaborator

bors commented Oct 26, 2020

⌛ Testing commit e86664e with merge 62b7ff53457c3abeffb95ac03b6676506593ca77...

@bors
Copy link
Collaborator

bors commented Oct 26, 2020

💔 Test failed - checks-actions

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Oct 26, 2020
@jonas-schievink
Copy link
Contributor

I guess this should be closed?

@Dylan-DPC-zz Dylan-DPC-zz deleted the rollup-3rs2czs branch October 26, 2020 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

InstCombine introduces an incorrect use of a local after its storage has ended